Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add servicing build targeting pack skip config #230

Closed
wants to merge 3 commits into from

Conversation

dagood
Copy link
Member

@dagood dagood commented Nov 11, 2019

For https://github.com/dotnet/core-setup/issues/8735. Uses dotnet/arcade#4318.

This sets up the WindowsDesktop targeting pack to only get built in stable builds if there's an intentional patch release. For 3.0/3.1, this is being done in Core-Setup:

3.0: dotnet/core-setup#8827
3.1: dotnet/core-setup#8828

@dagood
Copy link
Member Author

dagood commented Nov 12, 2019

@MichaelSimons @vatsan-madhavan @zsd4yr PTAL and merge if ok. I locked myself out when I renounced my adminity. 🙂 Might be reasonable to give dotnet-corefx write/contrib permissions?

@vatsan-madhavan
Copy link
Member

I'm not an expert in this but lgtm.

@dagood
Copy link
Member Author

dagood commented Nov 12, 2019

If there's someone in particular on the WPF or WinForms teams who handles (will be handling) servicing build version updates, it would be nice to have them take a look at this to make sure the workflow at least makes sense.

@vatsan-madhavan
Copy link
Member

/cc @RussKie, @AdamYoblick

@RussKie
Copy link
Member

RussKie commented Nov 12, 2019

FYI @zsd4yr has left the WinForms team, please don't tag him any longer.

@dagood
Copy link
Member Author

dagood commented Jan 14, 2020

Closing, dotnet/runtime#639 tracks a known issue with the current impl and merging a fixed version into both dotnet/runtime and dotnet/windowsdesktop master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants